Implemented deleteImage API, state and tests #945

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

gabrieltmonkai wants to merge 1 commit into main from feat/MN-780/create-deleteImage-api

Contributor

gabrieltmonkai commented Oct 8, 2025 •

edited

Loading

Overview

Jira Ticket Reference : MN-780

Added a new deleteImage function to handle image deletion of an inspection.

Checklist before requesting a review

I have updated the unit tests based on the changes I made
I have updated the docs (TSDoc / README / global doc) to reflect my changes
I have updated the local app configs if needed
I have performed self-QA of my feature by testing the apps and packages and made sure that :
- No regression or new bug has occurred
- The acceptance criteria listed in the ticket are met
- Self-QA was made on both desktop and mobile

gabrieltmonkai requested a review from dlymonkai

October 8, 2025 15:19

dlymonkai reviewed

View reviewed changes

packages/common/src/state/actions/deletedOneImage.ts Show resolved Hide resolved

gabrieltmonkai force-pushed the feat/MN-780/create-deleteImage-api branch from 3ab10ff to 4121c7f Compare

October 9, 2025 14:14

dlymonkai reviewed

View reviewed changes

packages/common/src/state/actions/deletedOneImage.ts Outdated Show resolved Hide resolved

gabrieltmonkai force-pushed the feat/MN-780/create-deleteImage-api branch 7 times, most recently from 0788148 to cf49a7c Compare

October 16, 2025 10:42

gabrieltmonkai force-pushed the feat/MN-780/create-deleteImage-api branch from cf49a7c to e8c1f0d Compare

October 20, 2025 08:59

dlymonkai reviewed

View reviewed changes

packages/network/README.md Outdated Show resolved Hide resolved

dlymonkai reviewed

View reviewed changes

packages/network/README.md Outdated Show resolved Hide resolved

dlymonkai reviewed

View reviewed changes

packages/network/src/api/image/requests.ts Show resolved Hide resolved

dlymonkai reviewed

View reviewed changes

packages/network/src/api/image/requests.ts Show resolved Hide resolved

dlymonkai reviewed

View reviewed changes

packages/network/src/api/image/requests.ts Outdated Show resolved Hide resolved

dlymonkai reviewed

View reviewed changes

packages/network/src/api/image/requests.ts Outdated Show resolved Hide resolved

dlymonkai reviewed

View reviewed changes

packages/common/src/state/actions/deletedOneImage.ts Show resolved Hide resolved

gabrieltmonkai force-pushed the feat/MN-780/create-deleteImage-api branch 3 times, most recently from 4af7b00 to b6d4572 Compare

October 21, 2025 08:59

dlymonkai reviewed

View reviewed changes

packages/network/README.md Outdated Show resolved Hide resolved

dlymonkai reviewed

View reviewed changes

packages/network/test/api/image/requests.test.ts Outdated Show resolved Hide resolved

gabrieltmonkai force-pushed the feat/MN-780/create-deleteImage-api branch 2 times, most recently from d864b1c to 6e0b9b2 Compare

October 21, 2025 15:19

dlymonkai reviewed

View reviewed changes

packages/network/test/api/image/requests.test.ts Outdated Show resolved Hide resolved

dlymonkai reviewed

View reviewed changes

packages/network/test/api/image/requests.test.ts Outdated Show resolved Hide resolved

dlymonkai reviewed

View reviewed changes

packages/network/test/api/image/requests.test.ts Outdated Show resolved Hide resolved


          Implemented deleteImage API, state and tests

aaec99e

Reset state objects containing the deleted image

Updated objects to be deleted in State

Updated API export and added more tests

Added docs

Updated Tests

gabrieltmonkai force-pushed the feat/MN-780/create-deleteImage-api branch from 6e0b9b2 to aaec99e Compare

October 22, 2025 15:01

dlymonkai assigned gabrieltmonkai

dlymonkai approved these changes

View reviewed changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet